Skip to content

Conversation

@jesgum
Copy link
Contributor

@jesgum jesgum commented Jan 9, 2026

@njacazio this PR has the following:

  • New decayer class that does most of the common decays (can be easily be expanded upon) using TDatabasePDG. We should also add a way to accept settings from a configuration file.
  • Task that decays short-lived particles and fills a new mcparticle table with the produced decay daughters. Here I think that the datamodel needs to be looked at. I didn't manage to make the table as a new version because there seem to be some issue when using expression columns when the table produced is used directly in another task, in the same workflow. But I think that this is OK for now as a starting point
  • QA task for the decayer

This workflow currently exists separately from the rest of the OTF machinery

@github-actions
Copy link

github-actions bot commented Jan 9, 2026

O2 linter results: ❌ 16 errors, ⚠️ 1 warnings, 🔕 0 disabled

@github-actions github-actions bot changed the title First iteration of OTF decayer [ALICE3] First iteration of OTF decayer Jan 9, 2026
Copy link
Collaborator

@njacazio njacazio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @jesgum

@njacazio njacazio merged commit e5aefca into AliceO2Group:master Jan 12, 2026
12 of 13 checks passed
@jesgum jesgum deleted the decayer branch January 12, 2026 15:38
@vkucera
Copy link
Collaborator

vkucera commented Jan 19, 2026

@jesgum The ALICE3/Core/Decayer.h file has plenty of unused/missing headers and uninitialised variables and members. Please check your code with code analysers before you make a PR.

@vkucera
Copy link
Collaborator

vkucera commented Jan 19, 2026

@jesgum The ALICE3/Core/Decayer.h file has plenty of unused/missing headers and uninitialised variables and members. Please check your code with code analysers before you make a PR.

Same for the other files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants